Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR is a continuation of the work on #3483 to update the NTD validation models for 2024 and next years:
all_ntdreports
.fct_ntd_rr20_service_checks
to verify if the values are not NULL or Zero before the calculation, and fixed ROUND function to use only at the end of the calculation.value_checked
results: Replaced NULL values by " " (space) to make the Concatenation function to return the existing values onfct_ntd_rr20_service_checks
,fct_ntd_a30_vomscheck
.int_ntd_a30_voms_vins_totals
year_of_data
column onfct_ntd_a30_vomscheck
Note: As much as I want to fix indentation and some patterns on those files, I couldn't change much because it was making hard to see the real changes (GIT DIFF).
Type of change
How has this been tested?
Tested locally creating tables on Staging:
cal-itp-data-infra-staging.external_blackcat.all_ntdreports
cal-itp-data-infra-staging.erika_mart_ntd_validation.fct_ntd_rr20_service_checks
cal-itp-data-infra-staging.erika_mart_ntd_validation.fct_ntd_a30_vomscheck
cal-itp-data-infra-staging.erika_staging.int_ntd_a30_voms_vins_totals
Post-merge follow-ups
Review the fixes on production after the next DAG
transform_warehouse
runs.cal-itp-data-infra.external_blackcat.all_ntdreports
cal-itp-data-infra.mart_ntd_validation.fct_ntd_rr20_service_checks
cal-itp-data-infra.mart_ntd_validation.fct_ntd_a30_vomscheck
cal-itp-data-infra.staging.int_ntd_a30_voms_vins_totals